Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Front-end Challenge - Edgar Dias #31

Open
wants to merge 46 commits into
base: master
Choose a base branch
from
Open

Conversation

EDGI9
Copy link

@EDGI9 EDGI9 commented Jan 28, 2023

What was implemented:

  • Use of Vue 2 and its features
  • Implemented styling to replicate provided design in the challenge, without the use of external UI libraries
  • Use of http request service to Fetch and Post the data to the same API provided in the challenge*
  • SEO meta information for further accessibility
  • Application responsiveness
  • Unit tests (Followed Test Driven Development approach)
  • Vuex to handle the data within the Vue APP
  • Vue-Router with just base routing but open for future expansion

*Remarks:

  • The challenge provides only one API Mocky endpoint and states to use it for both Fetching and Posting. When doing the post request it would be expected that that list of transactions would be updated but it returns always the same list. I researched briefly the Mocky documentation and didn't find information about posting requests. I hope this is the intended expectation and that the post request served no other purpose other that checking if it contained the correct payload when sent.

EDGI9 added 30 commits January 25, 2023 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant